Skip to content

Restore truncated imports in NLU test modules#73

Draft
Copilot wants to merge 2 commits into
mainfrom
copilot/arch-gap-fix-python-parse-errors-again
Draft

Restore truncated imports in NLU test modules#73
Copilot wants to merge 2 commits into
mainfrom
copilot/arch-gap-fix-python-parse-errors-again

Conversation

Copy link
Copy Markdown

Copilot AI commented Apr 5, 2026 •
edited
Loading

Description

The NLU test suite had two unparsable modules caused by truncated import blocks, which blocked collection and static analysis. This PR restores the missing imports in those files only; test behavior is otherwise unchanged.

  • tests/nlu_nlg/nlu/test_pipeline.py

    • Completed the import of NLUPipeline, NLUResult, and create_nlu_pipeline
    • Removed stray unused lines left inside the broken import block

  • tests/nlu_nlg/nlu/test_lexical_analyzer_parser.py

    • Completed the import of LexicalAnalyzerParser, SyntacticParseOutput, and Token
    • Removed stray unused lines left inside the broken import block

  • Example

    from godelOS.nlu_nlg.nlu.pipeline import (
    NLUPipeline, NLUResult, create_nlu_pipeline
)

from godelOS.nlu_nlg.nlu.lexical_analyzer_parser import (
    LexicalAnalyzerParser, SyntacticParseOutput, Token
)

Related Issues

Test Evidence

  • python -m py_compile tests/nlu_nlg/nlu/test_pipeline.py tests/nlu_nlg/nlu/test_lexical_analyzer_parser.py
  • python -m pytest --co -q tests/nlu_nlg/nlu/test_pipeline.py tests/nlu_nlg/nlu/test_lexical_analyzer_parser.py → 11 tests collected
  • python -m pytest -q tests/nlu_nlg/nlu/test_pipeline.py tests/nlu_nlg/nlu/test_lexical_analyzer_parser.py → 11 passed
  • Full repo collection now gets past these parse failures; remaining collection errors are unrelated pre-existing aiohttp import failures elsewhere

Checklist

  • Tests pass locally (pytest tests/)
  • Code is formatted (black . and isort .)
  • Documentation updated (if applicable)
  • No secrets or credentials committed
  • Related issue linked above

Copilot AI linked an issue Apr 5, 2026 that may be closed by this pull request
[arch-gap] Fix Python parse errors in 2 file(s) #72
Open
3 tasks
Copilot AI changed the title [WIP] Fix Python parse errors in two files Restore truncated imports in NLU test modules Apr 5, 2026
Copilot AI requested a review from Steake April 5, 2026 02:50
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 5, 2026

🧪 CI — Python 3.11

�[31mERROR�[0m tests/test_cognitive_subsystem_activation.py::�[1mTestEndToEndFlow::test_context_engine_round_trip�[0m - ModuleNotFoundError: No module named 'requests'
�[31mERROR�[0m tests/test_cognitive_subsystem_activation.py::�[1mTestEndToEndFlow::test_nlg_pipeline_process�[0m - ModuleNotFoundError: No module named 'requests'
�[31m===== �[31m�[1m4 failed�[0m, �[32m1277 passed�[0m, �[33m81 skipped�[0m, �[33m65 warnings�[0m, �[31m�[1m22 errors�[0m�[31m in 43.77s�[0m�[31m ======�[0m

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 5, 2026

🧪 CI — Python 3.10

�[31mERROR�[0m tests/test_cognitive_subsystem_activation.py::�[1mTestEndToEndFlow::test_context_engine_round_trip�[0m - ModuleNotFoundError: No module named 'requests'
�[31mERROR�[0m tests/test_cognitive_subsystem_activation.py::�[1mTestEndToEndFlow::test_nlg_pipeline_process�[0m - ModuleNotFoundError: No module named 'requests'
�[31m===== �[31m�[1m4 failed�[0m, �[32m1277 passed�[0m, �[33m81 skipped�[0m, �[33m65 warnings�[0m, �[31m�[1m22 errors�[0m�[31m in 44.91s�[0m�[31m ======�[0m

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Steake Steake Awaiting requested review from Steake Steake is a code owner

Assignees

@Steake Steake

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[arch-gap] Fix Python parse errors in 2 file(s)

2 participants

@Steake